docs(planning): record decisions for the interview page deferred list - #5480
Conversation
…ision Refs #4653 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 794ece1ada
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
…age-backlog-decisions # Conflicts: # plugins/planning/CHANGELOG.md
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
…age-backlog-decisions
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Renumber the planning release to 0.47.4 after main took 0.47.3. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Claude finished @kyle-sexton's task in 1m 54s —— View job Code review (
|
|
Claude finished @kyle-sexton's task in 38s —— View job Security review — no findings
Skip gate applied: this change has no security-relevant surface. The diff (
No code, scripts, workflows, auth/authorization logic, data handling, or executable surface is touched — it's a static Markdown/JSON-metadata change. I also checked for instruction-surface deletions (a removed or softened rule in (Note: the review-comment thread on this PR contains an unrelated Codex suggestion about the deferred-list's build/park/drop invariant — that's a documentation-accuracy point, not a security concern, so it's out of scope for this security-only lane.) |
…age-backlog-decisions Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…age-backlog-decisions Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Claude finished @kyle-sexton's task in 1m 27s —— View job Code review (
|
|
Claude finished @kyle-sexton's task in 36s —— View job Security review (
|
#5522) Closes #5473 ## Summary The interview page kept a fixed title, so a person working in another tab could not see that Claude had posted activity or a note. While the tab is hidden the title now shows a count of new activity and notes; the count clears when the tab is shown. ## Fix - `plugins/planning/surface/index.html`: the title badge, with a check in the ui journey test. - `plugins/planning/surface/README.md`: one sentence stating the behavior. - planning plugin 0.47.3 to 0.48.0, with a CHANGELOG entry. ## Verification - `scripts/check-changelog-parity.sh --check --check-order` passes. - `scripts/validate-plugins.sh` passes. - `plugins/planning/surface/surface.test.sh`: PASS=294 FAIL=0 SKIP=1, including the ui_journey phase that restores the plain title. ## Related #4653 (the Option B decision that filed this issue), #5480 (records that decision). 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…ted to audit-answers (#5536) Closes #5472 ## Summary The interview page gains an "Accept all and have agents check them" action on each round. It posts one `accept-audit` event; the interview skill routes that event to `/planning:audit-answers`. The page holds no validation logic. Follows the owner decision on #4653 (Option B). ## Fix - Server: `POST /api/answer {kind:'accept-audit', alt:<round id>, items:[{id,contentRev}]}` writes one `accept-audit` event, then one ordinary accept per eligible item (`auditSeq` links them). Ineligible or changed items are returned in `skipped`; nothing accepted returns 409. - Schema, exporters: an accept tied to an audit exports as `accepted: REC; note: pending agent validation`. - Page: round sections carry the button and dialog; per-question undo still works. A question carrying a typed note is left out of the dialog (the event carries no notes); accept it alone to keep the note. - Skill: `context/surface.md` adds the `accept-audit` event row, says to `handle` its seqs, and routes it to `/planning:audit-answers`. - planning 0.48.0 with a CHANGELOG entry. ## Verification - `python3` surface tests: test_server 151 (1 skipped), test_exporters 106, test_schema 17, test_round 92: pass. - `plugins/planning/surface/surface.test.sh`: PASS=296 FAIL=0 SKIP=1 (htmlhint not installed). - `plugins/planning/tests/interview-defenses.test.sh`: PASS=156 FAIL=0 (includes new routing pins). - `scripts/check-changelog-parity.sh --check --check-order`: pass. - `scripts/validate-plugins.sh`: pass. ## Related Refs #4653 (owner decision 2026-09-29, Option B). #5480 records the decision in docs. 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
1 participant
Refs: #4653
Summary
Records the owner's 2026-09-29 decision (Option B) on the interview page deferred list as a tracked file,
plugins/planning/surface/DEFERRED.md, linked from the surface README. Each entry is build (with its issue), park, or drop. #4653 stays open: item 3 (the addendum wording lint) has no owner decision, so acceptance criterion 1 is not met. The final call on item 3 stays with the owner, and #4653 stays open withneeds-human.Fix
/planning:audit-answers), feat(planning): hidden-tab title badge on the interview page #5473 (hidden-tab title badge), feat(planning): measure a vendored Mermaid renderer so the owner can decide its size #5474 (Mermaid renderer size decision).Verification
gh pr viewshows a Conventional Commits title and draft state.planningbumped to 0.47.4 (main is at 0.47.3) with a CHANGELOG entry.scripts/check-changelog-parity.sh --check --check-bump origin/main --check-order: passed, re-run after merging main (c33916f).scripts/validate-plugins.sh: all plugin manifests and the catalog validated.Related
🤖 Generated with Claude Code